Skip to content

Conversation

leemthompo
Copy link
Contributor

@leemthompo leemthompo commented Sep 30, 2025

Contributes to https://github.com/elastic/docs-content-internal/issues/334

I validated with a full docs-builder build locally

Summary

Created top-level contribute-docs/ section - New main section for public docs contribution guidelines
Moved API docs guidelines from extend/contribute/api-docs/ to contribute-docs/api-docs/
Updated docset.yml - Added toc: contribute-docs to main navigation
Updated extend/toc.yml - Removed references to moved files, simplified structure
Added redirects - All old URLs redirect to new locations
Created new TOC files - contribute-docs/toc.yml and contribute-docs/api-docs/toc.yml

Related PR in docs-builder

elastic/docs-builder#1969

@leemthompo leemthompo self-assigned this Sep 30, 2025
leemthompo added a commit to elastic/docs-builder that referenced this pull request Sep 30, 2025
Copy link
Contributor

@theletterf theletterf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGMT — thanks!

Only comment: how about renaming the index "Contribute to the docs" instead of "Contribute to docs"?

@leemthompo leemthompo marked this pull request as ready for review September 30, 2025 14:59
@leemthompo leemthompo requested review from a team as code owners September 30, 2025 14:59
@@ -0,0 +1,11 @@
toc:
- file: index.md
- file: api-docs/index.md
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • file: api-docs/index.md

This seems like a duplication of what's in the api-docs/toc.yml file. Is the duplication necessary?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good catch, I actually had a failing all-repo build when I didn't do this duplication

but maybe I could restructure somewhere else?

@leemthompo leemthompo requested review from a team and reakaleek October 2, 2025 10:49
@leemthompo leemthompo merged commit 3bddd62 into main Oct 9, 2025
8 checks passed
@leemthompo leemthompo deleted the leemthompo/contribute-docs-groundwork branch October 9, 2025 12:49
leemthompo added a commit to elastic/docs-builder that referenced this pull request Oct 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation Initiative 🚀

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants